-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
전남대 FE_김현채_1주차 과제 #32
base: main
Are you sure you want to change the base?
전남대 FE_김현채_1주차 과제 #32
Conversation
|
/** | ||
* Is this the principal call to action on the page? | ||
*/ | ||
primary: PropTypes.bool, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
primary 값은 불리언이 아니라 enumable 한 형태로 받는게 좋아보입니다.
eg) variant: "priamary" | "secondary"
|
||
interface InputProps extends React.InputHTMLAttributes<HTMLInputElement> { | ||
customSize?: 'small' | 'medium' | 'large' | 'responsive'; | ||
disabled?: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
disabled 는 기본적으로 가지는 어트리뷰트인데 따로 추가해준 이유가 궁금합니다.
안녕하세요. 멘토님! 전남대학교 김현채 입니다. 피드백을 위해 시간 내주셔서 감사합니다!
Q1. eslint를 추가하고 Lint 룰을 세팅한다는 개념에서 가장 어려움을 느꼈던 것 같습니다. 이 부분이 올바르게 설정된 건지 궁금합니다! 또한 멘토님의 Lint 룰 세팅 방법을 공유해주신다면 감사드립니다.
Q2. 폴더의 갯수를 늘려 기능 별로 파일들을 관리하는게 좀 더 이상적인 방법일까요? 검색을 통해 이상적인 폴더 관리에 대해 찾아보았는데 아직은 감이 잡히지 않는 것 같습니다..!
Q3. 커밋을 파일 한 개를 수정할 때마다(작은 단위로 꼼꼼히) 하는 것이 좋을지, 좀 더 유연하게 진행해도 좋을지 궁금합니다!(아직은 커밋을 하는게 쉽지 않아 놓치는 경우가 발생하여 여쭤보고 싶습니다.)
이외에도 잘못된 부분이나 수정이 필요한 부분을 알려주신다면 부족하지만 최대한 열심히 받아들이고 고쳐나가겠습니다!!